Skip to content

Conversation

iamriajul
Copy link

Description

Custom script to run after cloning the repository. Runs always after git clone, even if the repository already exists.

Type of Change

  • New module
  • Bug fix
  • Feature/enhancement
  • Documentation
  • Other

Module Information

Path: registry/coder/modules/git-clone
New version: v1.2.0
Breaking change: [ ] Yes [x] No

Testing & Validation

  • Tests pass (bun test)
  • Code formatted (bun run fmt)
  • Changes tested locally

Related Issues

@matifali matifali added the version:minor Add to PRs requiring a minor version upgrade label Aug 23, 2025
@DevelopmentCats
Copy link
Contributor

@iamriajul

These are from the logs of the failing test in git-clone

 Error: Expected: 128
  Received: 0
  
        at <anonymous> (/home/runner/work/registry/registry/registry/coder/modules/git-clone/main.test.ts:33:29)
  28 |     const state = await runTerraformApply(import.meta.dir, {
  29 |       agent_id: "foo",
  30 |       url: "fake-url",
  31 |     });
  32 |     const output = await executeScriptInContainer(state, "alpine/git");
  33 |     expect(output.exitCode).toBe(128);
                                   ^
  error: expect(received).toBe(expected)
  
  Expected: 128
  Received: 0
  
        at <anonymous> (/home/runner/work/registry/registry/registry/coder/modules/git-clone/main.test.ts:33:29)
  (fail) git-clone > runs with git [2524.00ms]

@DevelopmentCats
Copy link
Contributor

DevelopmentCats commented Sep 30, 2025

Once these are resolved I will test, and then we can merge

@DevelopmentCats
Copy link
Contributor

@iamriajul I just wanted to check in on this and see if there was an update?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
version:minor Add to PRs requiring a minor version upgrade
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants